Skip to content

API Clean-up (2) & Fixes #1868

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 14 commits into from
May 31, 2025
Merged

API Clean-up (2) & Fixes #1868

merged 14 commits into from
May 31, 2025

Conversation

Lord-Grey
Copy link
Collaborator

@Lord-Grey Lord-Grey commented May 19, 2025

Summary

  • Disable streaming cmds for HTTP/S

  • Align setEffect instance definition with standard

  • Clean-up subscriptions, ensure downward compatibility

  • Fixed Toggle-Buttons on Dashboard

  • Fixed Crash when XCB,X11,Qt grabber were configured and display manager changed to Wayland

  • Fixed Target-directory not correctly build when exporting effects

  • Fixed Effect not suspended when instance is stopped (Effect not suspended when instance is stopped #1586)

  • Fixed Background effect is started when instance is disabled

  • Fixed - GlobalSignals were not really global resulting input loss when toggling an instance off/on

  • Fixed Stopped instance objects were not destructed, as reference still exist

  • Added support for tracking creation/deletion of SmartPointer objects

  • Fixed Exit handling in main module

  • Updated Supported platforms

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Docs
  • Build-related changes
  • Other, please describe:

If changing the UI of web configuration, please provide the before/after screenshot:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

If yes, please describe the impact and migration path for existing setups:

The PR fulfills these requirements:

  • When resolving a specific issue, it's referenced in the PR's body (e.g. Fixes: #xxx[,#xxx], where "xxx" is the issue number)

If adding a new feature, the PR's description includes:

  • A convincing reason for adding this feature
  • Related documents have been updated (docs/docs/en)
  • Related tests have been updated

PLEASE DON'T FORGET TO ADD YOUR CHANGES TO CHANGELOG.MD

  • Yes, CHANGELOG.md is also updated

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.

Other information:

fixes #1586

@Lord-Grey Lord-Grey self-assigned this May 19, 2025
@Lord-Grey Lord-Grey added the Bug label May 19, 2025
@Lord-Grey Lord-Grey added this to Core May 19, 2025
@Lord-Grey Lord-Grey moved this to In Progress in Core May 19, 2025
@Lord-Grey Lord-Grey changed the title API Clean-up (2) API Clean-up (2) & Fixes May 20, 2025
@Lord-Grey Lord-Grey merged commit 37bf1cc into hyperion-project:master May 31, 2025
20 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Core May 31, 2025
@Lord-Grey Lord-Grey deleted the api branch May 31, 2025 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Effect not suspended when instance is stopped
1 participant